Module Safety plan status and role updates#447
Conversation
|
The created documentation from the pull request is available at: docu-html |
aschemmel-tech
force-pushed
the
aschemmel-tech-module-safety-status
branch
from
1d85851 to
144586d
Compare
|
A minor remark. The description of the Pull Request and the commit messages should describe what the change is about. This is described in https://eclipse-score.github.io/score/main/contribute/general/git.html#git-guidelines I would expect that this change would simply update the status of the module safety plan, but there are more code changes. Consider to describe in the PR heading and description what is contained. |
aschemmel-tech
force-pushed
the
aschemmel-tech-module-safety-status
branch
2 times, most recently
from
beaa47a to
367c737
Compare
aschemmel-tech
changed the title
aschemmel-tech
force-pushed
the
aschemmel-tech-module-safety-status
branch
from
367c737 to
9d281d6
Compare
aschemmel-tech
requested review from
PandaeDo,
PhilipPartsch,
masc2023 and
pahmann
as code owners
process/process_areas/process_management/process_management_workflow.rst
Show resolved
Hide resolved
There was a problem hiding this comment.
removed this
There was a problem hiding this comment.
example, e.g. for programming languages used, C++ or Rust
There was a problem hiding this comment.
reformulated
There was a problem hiding this comment.
Do we have defined FFI rated libraries somewhere?
There was a problem hiding this comment.
ok, removed FFI
There was a problem hiding this comment.
Same here wp__tailoring_safety, as we may have others, e.g. security
There was a problem hiding this comment.
renamed to wp_safety_tailoring
masc2023
left a comment
masc2023
left a comment
There was a problem hiding this comment.
See comments, at least tools should be stated as examples only
4 participants
Resolves: #78
Related to: eclipse-score/score#2325